From: Jasper St. Pierre Date: Wed, 18 Sep 2013 23:05:42 +0000 (-0400) Subject: overviewControls: Remove dead code X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~3137 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=1240d6be769f3791fa6933652bea259e6f05d4af;p=gnome-shell.git overviewControls: Remove dead code The parent SlidingControl had an onOverviewShowing, but we had overridden it with the same code in both subclasses. Just move it back to SlidingControl. --- diff --git a/js/ui/overviewControls.js b/js/ui/overviewControls.js index 00007f22a..fbf982f22 100644 --- a/js/ui/overviewControls.js +++ b/js/ui/overviewControls.js @@ -168,11 +168,10 @@ const SlidingControl = new Lang.Class({ }, _onOverviewShowing: function() { - // reset any translation and make sure the actor is visible when - // entering the overview this.visible = true; this.layout.slideX = this.getSlide(); - this.actor.translation_x = 0; + this.actor.translation_x = this._getTranslation(); + this.slideIn(); }, _onWindowDragBegin: function() { @@ -273,13 +272,6 @@ const ThumbnailsSlider = new Lang.Class({ return alwaysZoomOut; }, - _onOverviewShowing: function() { - this.visible = true; - this.layout.slideX = this.getSlide(); - this.actor.translation_x = this._getTranslation(); - this.slideIn(); - }, - getNonExpandedWidth: function() { let child = this.actor.get_first_child(); return child.get_theme_node().get_length('visible-width'); @@ -340,13 +332,6 @@ const DashSlider = new Lang.Class({ return 0; }, - _onOverviewShowing: function() { - this.visible = true; - this.layout.slideX = this.getSlide(); - this.actor.translation_x = this._getTranslation(); - this.slideIn(); - }, - _onWindowDragBegin: function() { this.fadeHalf(); },